feat(runtime): add managed DIRECT worker authority - #152
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A4.1b.1 scope
This Draft PR cuts over only newly admitted, deterministic DIRECT Runs in CI/test environments from legacy authority to the persisted managed Runtime authority selected by A4.1a. Worker routing uses the immutable
TaskRun.runtime_authority; it never re-evaluates the admission gate and never falls back to the legacy runner.Authority and transaction changes
RECONCILIATION_REQUIREDand AttemptOUTCOME_UNKNOWNstates with no ordinary exit.DISPATCHINGin short transactions, with all adapter/provider calls outside database transactions.Crash windows and recovery
reattach=falseis never redispatched. The old owner/fence records deterministic control-plane uncertainty and parks Runtime, Task, Run, and Attempt atomically.Rollout and rollback
managed_runtime_direct_cutoverremains absent from default profiles, restricted to test/testing with the deterministic provider, and disabled on the test server. Disabling it changes admission for new Runs only. Existing managed Runs must retain the managed worker/runtime configuration until terminal or parked; rewriting authority or falling back to legacy is prohibited.Validation
Local:
git diff --check: passedGitHub PostgreSQL CI provides real transaction evidence for:
Out of scope